Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update build.yml to fix build for armv5 without fpu #1459

Open
wants to merge 40 commits into
base: Meta
Choose a base branch
from

Conversation

zusterben
Copy link

test build :CGO_ENABLED=0 GOOS=linux GOARCH=arm GOARM=5,softfloat go build -tags with_gvisor -ldflags "-w -s"
test results:Mihomo Meta 1.10.0 linux arm with go1.22.1 unknown time Use tags: with_gvisor
test cpu:bcm4709
Now, it runs fine on armv5 without fpu.

adlyq and others added 30 commits June 9, 2022 14:18
add permissions for systemctl services
clash-dashboard change to updated one
# Conflicts:
#	rules/provider/classical_strategy.go
# Conflicts:
#	.github/workflows/docker.yaml
#	.github/workflows/prerelease.yml
@wwqgtxx
Copy link
Collaborator

wwqgtxx commented Aug 19, 2024

@zusterben 根据golang文档
GOARM=5 defaults to softfloat and GOARM=6 and 7 default to hardfloat.
所以对于GOARM=5应该不需要手动设置softfloat

@zusterben
Copy link
Author

@wwqgtxx 但实测,armv5是用的hardfloat,原因不明,现在强制指定softfloat至少能保证正常运行

@wwqgtxx
Copy link
Collaborator

wwqgtxx commented Aug 19, 2024

你有尝试在最新的go1.23.0下GOARM=5GOARM=5,softfloat下表现是否有区别么

@zusterben
Copy link
Author

问题不在go版本,因为现在的版本我也编译其他项目,是正常默认softfloat的

@wwqgtxx
Copy link
Collaborator

wwqgtxx commented Aug 19, 2024

本机在1.23.0下测试了GOARM=5GOARM=5,softfloat,编译出来的二进制文件是一模一样的

@zusterben
Copy link
Author

更正,其他项目的armv5在1.22版本下也是默认hardfloat

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants